Revert 15164:d93e560c1d50afcf39c99a4211c20c8b9ce63c38
authorkfraser@localhost.localdomain <kfraser@localhost.localdomain>
Fri, 25 May 2007 12:41:01 +0000 (13:41 +0100)
committerkfraser@localhost.localdomain <kfraser@localhost.localdomain>
Fri, 25 May 2007 12:41:01 +0000 (13:41 +0100)
tools/python/xen/xend/XendDomainInfo.py

index 3845a7ddade9bdb1040573ed30f3e93e19c19d27..ca780c2895e415dea4bd284c499fac779b31dfdf 100644 (file)
@@ -362,7 +362,6 @@ class XendDomainInfo:
         self.vmWatch = None
         self.shutdownWatch = None
         self.shutdownStartTime = None
-        self.unresponsive = False
         self._resume = resume
 
         self.state_updated = threading.Condition()
@@ -1129,25 +1128,21 @@ class XendDomainInfo:
                 # failed.  Ignore this domain.
                 pass
             else:
-                # Domain is alive.
+                # Domain is alive.  If we are shutting it down, then check
+                # the timeout on that, and destroy it if necessary.
                 if xeninfo['paused']:
                     self._stateSet(DOM_STATE_PAUSED)
                 else:
                     self._stateSet(DOM_STATE_RUNNING)
                     
-                if self.shutdownStartTime and not self.unresponsive:
+                if self.shutdownStartTime:
                     timeout = (SHUTDOWN_TIMEOUT - time.time() +
                                self.shutdownStartTime)
                     if timeout < 0:
-                        # The domain is not responding to shutdown requests.
-                        # Log a message, and rename the domain to indicate the
-                        # state; we keep the domain running, however, to
-                        # allow corrective action.
                         log.info(
                             "Domain shutdown timeout expired: name=%s id=%s",
                             self.info['name_label'], self.domid)
-                        self.setName('unresponsive-' + self.getName())
-                        self.unresponsive = True
+                        self.destroy()
         finally:
             self.refresh_shutdown_lock.release()
 
@@ -1436,7 +1431,6 @@ class XendDomainInfo:
         log.debug('XendDomainInfo.constructDomain')
 
         self.shutdownStartTime = None
-        self.unresponsive = False
 
         hvm = self.info.is_hvm()
         if hvm: